Skip to content

Initial introduction of exceptions to requests.#48

Draft
matejak wants to merge 1 commit intoEnterpriseyIntranet:masterfrom
matejak:exceptions
Draft

Initial introduction of exceptions to requests.#48
matejak wants to merge 1 commit intoEnterpriseyIntranet:masterfrom
matejak:exceptions

Conversation

@matejak
Copy link
Copy Markdown

@matejak matejak commented Dec 16, 2019

No description provided.

@scrutinizer-notifier
Copy link
Copy Markdown

scrutinizer-notifier commented Dec 16, 2019

The inspection completed: 1 new issues, 11 updated code elements

@codecov-io
Copy link
Copy Markdown

codecov-io commented Dec 16, 2019 •
edited
Loading

Codecov Report

Merging #48 into master will decrease coverage by 2.18%.
The diff coverage is 64%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
- Coverage   92.71%   90.52%   -2.19%     
==========================================
  Files          17       17              
  Lines         563      623      +60     
==========================================
+ Hits          522      564      +42     
- Misses         41       59      +18
Impacted Files Coverage Δ
src/nextcloud/api_wrappers/apps.py 100% <100%> (ø) ⬆️
src/nextcloud/base.py 76% <60%> (-24%) ⬇️
src/nextcloud/api_wrappers/user_ldap.py 100% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3253a7c...60a099d. Read the comment docs.

@matejak matejak mentioned this pull request Feb 6, 2020
Open
@luffah
Copy link
Copy Markdown

luffah commented Apr 14, 2021

Note : This PR don't solve #5.

Maybe the PR will benefit of filling the description (with API reference source) and maybe associating it to a specific issue report.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 2, 2026

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 64.00000% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.52%. Comparing base (3253a7c) to head (60a099d).
⚠️ Report is 13 commits behind head on master.

Files with missing lines Patch % Lines
src/nextcloud/base.py 60.00% 18 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files 
@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
- Coverage   92.71%   90.52%   -2.19%     
==========================================
  Files          17       17              
  Lines         563      623      +60     
==========================================
+ Hits          522      564      +42     
- Misses         41       59      +18

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@matejak @scrutinizer-notifier @codecov-io @luffah @codecov-commenter